Skip to content

Conversation

@Jonathing
Copy link
Member

Removes dependency on Log Utils since it really doesn't need it. Also, moved the multi-version specific code to DownloadUtilsImpl so that the entire class doesn't have to be copied between two different projects.

@Jonathing Jonathing requested a review from LexManos October 25, 2025 03:25
@Jonathing Jonathing self-assigned this Oct 25, 2025
import java.util.ArrayList;
import java.util.List;

public final class DownloadUtilsImpl {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Package private

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops

@Jonathing Jonathing force-pushed the feat/jonathing/download-utils/0.4 branch 2 times, most recently from d851083 to 6b45c51 Compare October 25, 2025 21:17
@Jonathing Jonathing force-pushed the feat/jonathing/download-utils/0.4 branch from 6b45c51 to 8eaf86f Compare October 25, 2025 21:17
@Jonathing
Copy link
Member Author

Merging this now so I can get started updating other things that use this. If it breaks, just blame it on me and I'll fix it. Minor version bump.

@Jonathing Jonathing merged commit 8eaf86f into MinecraftForge:master Oct 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants